Skip to content

pynng: add build-pynng.yml for riscv64 wheels - #1454

Merged
luhenry merged 2 commits into
mainfrom
pynng
Sep 8, 2026
Merged

pynng: add build-pynng.yml for riscv64 wheels#1454
luhenry merged 2 commits into
mainfrom
pynng

Conversation

@luhenry

@luhenry luhenry commented Sep 7, 2026

Copy link
Copy Markdown
Member

Builds pynng's CFFI binding to nng, which statically links nng and mbedtls (both cloned and compiled from source at build time by setup.py). Upstream publishes no riscv64 wheel.

Mirrors upstream's cibuildwheel.yml.

Differs from upstream

  • Drops musllinux - cffi, a build and runtime dependency, has no musllinux riscv64 wheel on our registry.

Testing

  • same as upstream

License: Wheel bundles nng (MIT) and mbedtls (Apache-2.0 OR GPL-2.0-or-later); upstream's wheel carries only its own LICENSE, so the build stages LICENSE.nng and LICENSE.mbedtls too.

luhenry added a commit that referenced this pull request Sep 7, 2026
pyproject.toml's classic license = {file = "LICENSE.txt"} suppresses
setuptools' default LICEN[CS]E* glob, so LICENSE.nng/LICENSE.mbedtls
staged by the workflow were silently dropped from the wheel's licence
metadata (the cp312/cp314/cp314t CI jobs caught this: the wheel shipped
none of the three license files at all). Patch pyproject.toml to declare
license-files explicitly, and fix the CIBW_TEST_COMMAND assertion, which
had assumed a dist-info/licenses/ subfolder this setuptools version
does not use.
@luhenry
luhenry merged commit 8ce5906 into main Sep 8, 2026
9 checks passed
@luhenry
luhenry deleted the pynng branch September 8, 2026 08:36
@luhenry luhenry mentioned this pull request Sep 8, 2026
@luhenry luhenry linked an issue Sep 8, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pynng riscv64 support

1 participant